Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mockup for gTTS support (see #59) #60

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

GijsTimmers
Copy link

See #59.

Everything works as it should, but the generated wavefile doesn't play on the iPod.

Do we need to encode it in a certain way with ffmpeg?

@GijsTimmers
Copy link
Author

Actually it does work as-is. I was pointing the script to the wrong directory. Please review and merge.

Copy link
Collaborator

@NicoHood NicoHood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Seems straight forward.

@NicoHood
Copy link
Collaborator

@nims11 your feedback?

@@ -124,6 +124,14 @@ def check_support():
else:
voiceoverAvailable = True

# Check for gtts-cli voiceover
# https://github.com/pndurette/gTTS
if not exec_exists_in_path("gtts-cli"):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh wait, we need to check for ffmpeg here as well, right?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dont we also need to check for sox of rhvoice? This is not related to this PR, but I just found this there as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants